Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embed original traceback in Hydra's exception report. #2863

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

gleize
Copy link
Contributor

@gleize gleize commented Mar 1, 2024

Motivation

When Hydra runs jobs (multirun mode), and the job crashes, only the exception gets reported and raised (but not the original traceback). This is missing for proper / faster debugging of the original error.
This PR solves that missing feature.

Have you read the Contributing Guidelines on pull requests?

No

Test Plan

Tested with the submitit plugin, which pickles the exception information properly.

Related Issues and PRs

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants